-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CRAN release documentation update #82
Conversation
@ramiromagno @galachad - This PR is ready for your review. Let's merge it before we create the release candidate. If there are any pipeline errors, please feel free to fix them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rammprasad and @galachad: I've made a few changes:
- Whitelisted some acronyms that got introduced with the new docs
- Fixed the
VignetteIndexEntry
value to match the vignette title in study_sdtm_spec.Rmd - Fixed the Slack URL that was incomplete in README.md
- Added my name and affiliation to the authors' section to pkgdown configuration for side nav bar hyperlinking
- Rerun current package state in linux, macos, macos-arm64, windows VMs with rhub which seems to be passing fine.
Co-authored-by: edgar-manukyan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Documentation updates
closes #81